-
Notifications
You must be signed in to change notification settings - Fork 124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix/esp lvgl port (BSP-630) #496
Conversation
7e8e82d
to
450d1ec
Compare
450d1ec
to
9dd9bb3
Compare
9dd9bb3
to
099b7e3
Compare
@100312dog Could you please split this PR into 2? One for s3-eye BSP and one for esp_lvgl_port? The changes in esp_lvgl_port LGTM, and I don't want to delay it anymore |
099b7e3
to
3aac591
Compare
@tore-espressif OK, it's done. |
@espzav PTAL, it seems that we allocated buffers that were too big for RGB565! |
Yes, it seems that it is true. I am sorry for this mistake, I bad understanded before. I think that Gabor told it and I forgot fix it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@100312dog Thank you for this fix. LGTM
3aac591
to
62c7ed1
Compare
ESP-BSP Pull Request checklist
Change description